New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor(theme): move LayoutProviders to Layout/Provider; composeProviders util #7676
Conversation
✅ [V2]
To edit notification comments on pull requests, go to your Netlify site settings. |
⚡️ Lighthouse report for the deploy preview of this PR
|
I think it's better this way. |
Ok, we can try this A little bit afraid it removes some flexibility in the end (like the ability to swap one of our provider for something else 🤷♂️ ), but not sure to have any good usecase so we'll see if anyone complains The |
Size Change: +65 B (0%) Total Size: 802 kB
ℹ️ View Unchanged
|
Ah sorry! I didn't make it clear. I meant I prefer the current way. (You know how much I don't like over-abstraction...) |
ah 😅 let's revert then. Anyway ejecting |
This reverts commit 25b8563.
Pre-flight checklist
Motivation
As in other places where we nest components, move
@theme/LayoutProviders
to@theme/Layout/Provider
Also use a new
composeProviders
fn to avoid the previous ugly nestingThinking of maybe making ThemeCommon expose a single
ThemeCommonProvider
? 🤔Test Plan
preview